Are you sure that layoutParams.screenBrightness is in range [0.0, 1.0]? The reference says:

A value of less than 0, the default, means to use the preferred screen brightness. 0 to 1 adjusts the brightness from dark to full bright.

Couldn't this go below 0?

Oh, #3102 is another PR that does the same thing, but it also prints some debug info. I'd use that one, what do you think @TobiGr ?

Are you sure that layoutParams.screenBrightness is in range [0.0, 1.0]? The reference says:

A value of less than 0, the default, means to use the preferred screen brightness. 0 to 1 adjusts the brightness from dark to full bright.

Couldn't this go below 0?

We do get negative value of -1 if the user has never used brightness gesture. But since we're not using any negative value for ProgressBar using setMin(), the default minimum is 0. So assigning a negative value will default to 0 in our case.

I guess we can set the gesture value to be whatever the system brightness is if the user has never used the gesture?

@Ullas-Aithal see #3102 (review)

@Stypox , oh, its the same change 😄. How does this work? Should I work on my pull request or wait for the author to work on #3102 ?

@Ullas-Aithal #3102 also adds debug information, so I think it is preferable. So you probably shouldn't work on this PR, sorry ;-)
Thank you very much for your contribution, anyway! I will leave the PR open just in case the creator of the other PR does not resolve the submitted reviews.

@Stypox that's totally fine. I'll probably take a look at another bug maybe. Thank you for maintaining this app!

@Ullas-Aithal I merged #3102 in the end. Thank you for your contribution, anyway! :-)